Skip to content

Conversation

ahmedmandur
Copy link
Contributor

@ahmedmandur ahmedmandur commented Dec 22, 2024

Added all missing properties from Flutter's ChoiceChip widget to FormBuilderChoiceChip to enhance customization options and match Flutter's native widget capabilities.

Connection with issue(s)

Connected to #1448

Solution description

Added the following properties from Flutter's ChoiceChip widget to FormBuilderChoiceChip:

Core properties:

  • surfaceTintColor: For Material 3 surface tint color support
  • clipBehavior: For controlling how the chip clips its content
  • checkmarkColor: For customizing the checkmark color when selected
  • autofocus: Control initial focus state
  • avatarBoxConstraints: Control avatar size and constraints
  • chipAnimationStyle: Customize selection animation style
  • color: State-based color customization
  • iconTheme: Icon styling customization
  • tooltip: Long-press tooltip support

All properties were added with their original documentation and default values to maintain consistency with Flutter's implementation.

Screenshots or Videos

N/A - The changes are property additions that enhance customization options without visual changes to the default appearance.

To Do

  • Read contributing guide
  • Check the original issue to confirm it is fully satisfied
  • Add solution description to help guide reviewers
  • Add documentation to code properties and package readme
  • Add unit test to verify new or fixed behaviour

Add surfaceTintColor, clipBehavior, and checkmarkColor properties from Flutter's ChoiceChip widget
@ahmedmandur ahmedmandur force-pushed the feature/add-missing-choicechip-properties branch from 3ea2a2e to bae4075 Compare December 22, 2024 13:42
@deandreamatias
Copy link
Collaborator

Hi, thanks for contribution!
If you can add this properties too, I will apreciated
Screenshot 2024-12-23 at 08 46 39

Add additional properties from Flutter's ChoiceChip widget:
- autofocus
- avatarBoxConstraints
- chipAnimationStyle
- color
- iconTheme
- tooltip

Update documentation in both code and README

Fixes flutter-form-builder-ecosystem#1448
@ahmedmandur
Copy link
Contributor Author

i made the changes and added the mentioned properties.
i also made some README.md format for better readability without changing the content.

@deandreamatias deandreamatias changed the title feat: add missing ChoiceChip properties feat: #1448 add missing ChoiceChip properties Dec 24, 2024
Copy link

codecov bot commented Dec 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.11%. Comparing base (08ccbc9) to head (7e229f8).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1449   +/-   ##
=======================================
  Coverage   88.11%   88.11%           
=======================================
  Files          21       21           
  Lines         774      774           
=======================================
  Hits          682      682           
  Misses         92       92           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@deandreamatias deandreamatias merged commit 1c9acc8 into flutter-form-builder-ecosystem:main Dec 24, 2024
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants